Skip to content

Conversation

isentropic
Copy link
Contributor

No description provided.

@ViralBShah ViralBShah changed the title Followup from https://github.com/JuliaLang/julia/pull/48507 (add news for Dates) Add news for Dates - ISO year / ISO week utilities Oct 13, 2025
@ViralBShah ViralBShah merged commit 1a8c862 into JuliaLang:master Oct 13, 2025
5 of 9 checks passed
@KristofferC
Copy link
Member

This didn't pass CI and seems to have broken the build?


┌ Error: invalid local link/image: file does not exist in src/NEWS.md
--
  | │   link =
  | │    @ast MarkdownAST.Link("ISO week date", "") do
  | │      MarkdownAST.Text("https://en.wikipedia.org/wiki/ISO")
  | │      MarkdownAST.Emph() do
  | │        MarkdownAST.Text("week")
  | │      end
  | │      MarkdownAST.Text("date")
  | │    end
  | │
  | └ @ Documenter /cache/build/builder-amdci4-4/julialang/julia-master/deps/jlutilities/depot/packages/Documenter/iRt2s/src/utilities/utilities.jl:47
  | [ Info: CheckDocument: running document checks.
  | [ Info: Populate: populating indices.
  | ERROR: LoadError: `makedocs` encountered an error [:cross_references] -- terminating build before rendering.
  | Stacktrace:

@KristofferC
Copy link
Member

I've reverted this

@isentropic
Copy link
Contributor Author

Is there anything to do on my end? (im the author of #48507)

@isentropic
Copy link
Contributor Author

This is probably because I ddint format the wiki link correctly right? How should I have done it?

@KristofferC
Copy link
Member

See https://anvilproject.org/guides/content/creating-links.

Also

`isoweekdate, isoyear, weeksinyear`

should be

`isoweekdate`, isoyear`, `weeksinyear`

@ViralBShah
Copy link
Member

CI was green when I merged it IIRC.

@adienes adienes added the reverted This PR has since been reverted label Oct 14, 2025
@KristofferC
Copy link
Member

KristofferC commented Oct 14, 2025

https://buildkite.com/julialang/julia-master/builds/51206#_ is what the last CI shows for me before it was merged and it also has a red cross.

image

But, easily fixed so 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reverted This PR has since been reverted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants